Conversation
Bump dependency versions including @babel/*, @apollo/client, graphql, @openfeature/web-sdk, uuid, svgo, chokidar, axios, and others. Clean up unused resolution overrides in root package.json.
There was a problem hiding this comment.
Pull request overview
Note
Copilot was unable to run its full agentic suite in this review.
Updates the monorepo’s JavaScript toolchain and runtime dependencies (Babel, Apollo/GraphQL, OpenFeature, chokidar, axios, uuid, etc.) and removes some no-longer-needed root resolutions overrides.
Changes:
- Bumped multiple workspace dependencies and refreshed
yarn.lockaccordingly. - Tightened/updated several package
peerDependenciesto newer minimum versions (notably Apollo/GraphQL). - Simplified root
package.jsonby removing a set ofresolutionsoverrides.
Reviewed changes
Copilot reviewed 6 out of 7 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| yarn.lock | Reflects updated dependency graph from bumped versions (Babel, OpenFeature, SWC, axios, chokidar, uuid, etc.). |
| package.json | Removes several resolutions entries and keeps a smaller set of overrides. |
| example/package.json | Updates example app dependencies (e.g., axios, openfeature react sdk). |
| packages/react-native-session-replay/package.json | Bumps chokidar major version. |
| packages/react-native-openfeature/package.json | Bumps OpenFeature SDK versions and peer range. |
| packages/react-native-babel-plugin/package.json | Updates Babel/SWC/TypeScript/Jest typings plus bumps uuid. |
| packages/react-native-apollo-client/package.json | Bumps Apollo/GraphQL dependencies and raises peer minimums. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| "resolutions": { | ||
| "cosmiconfig@npm:5.2.1/js-yaml": "3.14.2", | ||
| "front-matter@npm:4.0.2/js-yaml": "3.14.2", | ||
| "cacache/glob": "^10.5.0", | ||
| "@npmcli/arborist": "^9.1.8", | ||
| "ejs": "3.1.10", | ||
| "cross-spawn": "7.0.5", | ||
| "fast-xml-parser": "4.5.5", | ||
| "axios": "1.15.0", | ||
| "ajv@^8.12.0": "8.18.0", |
There was a problem hiding this comment.
The root resolutions still pins axios to 1.15.0, but the example app was bumped to axios: 1.15.1 and the lockfile shows axios 1.15.1. This can cause Yarn to force-install 1.15.0 unexpectedly (or produce churn in yarn.lock depending on install order). Consider either updating the resolution to 1.15.1 or removing the axios resolution entirely if it’s no longer required.
| "@swc/jest": "^0.2.38", | ||
| "@types/jest": "^29.5.14", | ||
| "@types/jest": "^30.0.0", | ||
| "jest": "^29.7.0", |
There was a problem hiding this comment.
@types/jest is bumped to ^30.0.0 while jest remains on ^29.7.0. This mismatch can introduce incorrect typings and also pulls Jest 30 runtime packages (e.g., expect@30.x) into the dependency graph via the typings package. Align the major versions by either keeping @types/jest on ^29.x or upgrading jest to ^30.x (and validating config/transform compatibility).
| "peerDependencies": { | ||
| "@apollo/client": ">=3.0", | ||
| "@apollo/client": ">=3.13.9", | ||
| "@datadog/mobile-react-native": "^3.0.0", | ||
| "graphql": ">=15.0.0", | ||
| "graphql": ">=16.13.2", | ||
| "react": ">=16.13.1", | ||
| "react-native": ">=0.63.4 <1.0" | ||
| }, |
There was a problem hiding this comment.
The peer dependency minimums were raised from @apollo/client >=3.0 to >=3.13.9 and from graphql >=15.0.0 to >=16.13.2. For published libraries, raising peer minimums is a breaking change for consumers pinned to older versions. If this package follows semver, this should usually be accompanied by a major version bump; alternatively, consider keeping broader peer ranges (while still using newer versions in devDependencies) unless there’s a hard requirement for the newer APIs.
What does this PR do?
Bumps dependency versions including @babel/*, @apollo/client, graphql, @openfeature/web-sdk, uuid, svgo, chokidar, axios, and others.
Cleans up unused resolution overrides in root package.json.
Review checklist (to be filled by reviewers)